licenses: document invite-time license assignment - #25678
Conversation
Admins can assign a product license when inviting a member, but the license page claimed inviting consumes a seat or license immediately. The pool is deducted on accept, and an invitee who accepts after the pool empties joins without a license and without any notification, so admins need to confirm available licenses before bulk invites. Co-authored-by: Cursor <cursoragent@cursor.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The changes cleanly document the new invite-time license assignment feature. The new section in manage-licenses.md is well-structured — it explains the workflow, the deferred-deduction behavior, and the edge case where licenses run out, all in clear, concise prose that matches Docker's style. The brief cross-reference paragraph added to members.md integrates smoothly. No AI-isms, no markdown issues, no missing redirects, no front-matter problems, no high or medium severity issues found.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Move invite-time license assignment under Licenses and invites, tighten the invite procedure and accept note, and point members.md at the new section anchor. Co-authored-by: Cursor <cursoragent@cursor.com>
Point the Docker Sandboxes link at /manuals/ai/sandboxes/_index.md so htmltest resolves the target. Co-authored-by: Cursor <cursoragent@cursor.com>
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR cleanly documents the invite-time license assignment feature. The new sections in both manage-licenses.md and members.md follow Docker style conventions: no hedge words, no "allows/enables you to" violations, correct use of "select" over "click", bold only on UI elements, and internal links correctly use the /manuals/ prefix with valid anchor slugs. The brief "Accept invites" section is appropriately scoped for the admin audience — it informs admins what the invitee must do next, rather than duplicating invitee-facing guidance. Scope and tone match the existing document character.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
craig-osterhout
left a comment
craig-osterhout
left a comment
There was a problem hiding this comment.
lgtm
+1 to the agent nit
ob1dev
left a comment
ob1dev
left a comment
There was a problem hiding this comment.
Looking good, please see my suggestions...
Clarify select-vs-assign timing, note the no-notification limitation, that licenses are not reserved for pending invites, multi-select licenses, auto-assignment availability, and accept via email or Notifications Center. Co-authored-by: Cursor <cursoragent@cursor.com>
Tighten intro prose, align members accept paths with Notifications Center, clarify multi-license selection on accept, and improve members SEO front matter. Keep export heading unchanged to preserve existing anchors. Co-authored-by: Cursor <cursoragent@cursor.com>
Rename the invite procedure to select licenses at invite time, and document Notifications Center acceptance steps alongside the email path. Co-authored-by: Cursor <cursoragent@cursor.com>
4 participants
Assign license on invite feature